Fix another issue of poisoning too eagerly
authorAlex Crichton <alex@alexcrichton.com>
Fri, 6 Apr 2018 19:35:37 +0000 (12:35 -0700)
committerAlex Crichton <alex@alexcrichton.com>
Fri, 6 Apr 2018 21:28:46 +0000 (14:28 -0700)
commit8fddd6fda525689a9073b2f32f823a9d0a0c1c2b
treeb7e8e4e638ea30653709eb53bba60f59b8fd62ee
parent0dabed904b739d8a6d1aee37070b10d60599d50a
Fix another issue of poisoning too eagerly

This commit extends the fix in #5288 by moving the logic added farther up in the
loop over package dependencies. This means that we won't recursively look at
optional/dev path dependencies which aren't members of the workspace. This
should fix the new issue that came up in #5257

Closes #5257
src/cargo/ops/resolve.rs
tests/testsuite/freshness.rs